Skip to content

CURATOR-596 Upgrade ZooKeeper to 3.7.0 - #385

Closed
eolivelli wants to merge 4 commits into
apache:masterfrom
eolivelli:CURATOR-596
Closed

CURATOR-596 Upgrade ZooKeeper to 3.7.0#385
eolivelli wants to merge 4 commits into
apache:masterfrom
eolivelli:CURATOR-596

Conversation

@eolivelli

@eolivelli eolivelli commented May 17, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@eolivelli
eolivelli requested review from Randgalt and cammckenzie May 17, 2021 13:17
@cammckenzie

cammckenzie commented May 18, 2021

Copy link
Copy Markdown
Contributor

Changes look fine to me, but it appears the build unit tests are failing on the CI build?

@eolivelli

eolivelli commented May 18, 2021

Copy link
Copy Markdown
Contributor Author

Yes.
Probably it is better to just update to latest 3.6.3 and move to 3.7.0 when I am done on #379

@eolivelli

eolivelli commented May 18, 2021

Copy link
Copy Markdown
Contributor Author

On the other hand people may try to use Curator with 3.7.0 and we must be sure that they do not have problems.
I will investigate the failures

@Hronom

Hronom commented Dec 13, 2021

Copy link
Copy Markdown

What the status?

@eolivelli

eolivelli commented Dec 14, 2021

Copy link
Copy Markdown
Contributor Author

@Hronom
thanks for the reminder !

I have resolved the conflicts, if CI passes I will commit the patch.

Maybe we can follow up with a release of Curator

@elkozmon elkozmon mentioned this pull request Dec 17, 2021
Open
@tommy04062019

tommy04062019 commented Feb 18, 2022

Copy link
Copy Markdown

Anything new here?I've waiting for this a long time

@tisonkun

tisonkun commented May 10, 2022

Copy link
Copy Markdown
Member

@eolivelli it seems there is still some test failure :(

@mariouso

mariouso commented Jun 1, 2022

Copy link
Copy Markdown

Hi @eolivelli is there any update on the curator with 3.7.1 zk version and the new testing server?

@eolivelli

eolivelli commented Jul 1, 2022

Copy link
Copy Markdown
Contributor Author

@tisonkun feel free to pick up this patch

@eolivelli

eolivelli commented Jul 1, 2022

Copy link
Copy Markdown
Contributor Author

we should move to 3.7.1

@tisonkun

tisonkun commented Jul 14, 2022

Copy link
Copy Markdown
Member

Resolved by #426.

@tisonkun tisonkun closed this Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cammckenzie cammckenzie Awaiting requested review from cammckenzie
@Randgalt Randgalt Awaiting requested review from Randgalt

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@eolivelli @cammckenzie @Hronom @tommy04062019 @tisonkun @mariouso